-
-
Notifications
You must be signed in to change notification settings - Fork 33k
Make sure :keyword:
role works for case
#138878
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Looks like that it is correct now: :keyword:`case` generates https://cpython-previews--138878.org.readthedocs.build/en/138878/reference/compound_stmts.html#case as a link. And it generates this HTML: <span id="case"></span>
<span id="match"></span> ![]() So, I am reverting the test change :) |
Thanks @sobolevn for the PR, and @AA-Turner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14. |
(cherry picked from commit 848d926) Co-authored-by: sobolevn <[email protected]>
(cherry picked from commit 848d926) Co-authored-by: sobolevn <[email protected]>
GH-139200 is a backport of this pull request to the 3.14 branch. |
GH-139201 is a backport of this pull request to the 3.13 branch. |
#139200) Make sure the ``:keyword:`` role works for ``case`` (GH-138878) (cherry picked from commit 848d926) Co-authored-by: sobolevn <[email protected]>
Thanks a lot to @AA-Turner for help.
I decided not to change the heading, because it can be a part of the existing links: https://docs.python.org/3/reference/compound_stmts.html#the-match-statement And we can break it.
I also added a quick test for
:keyword:case
, let's see :)📚 Documentation preview 📚: https://cpython-previews--138878.org.readthedocs.build/